🌤️ feat: Add OpenWeather Tool for Weather Data Retrieval #5246
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…addington/LibreChat into jmaddington-new/feature/openweather-clean
2 tasks
|
@jmaddington One thing I noticed is that this isn't setup for user-provided keys. Part of the logic involves I'll add it when I get the chance |
|
@danny-avila I'd be glad to provide a key, or we can skip the integration tests if the key doesn't exist. While mock tests are great, I personally have a high preference for live integration tests when it is feasible. |
|
All good, I meant that the current code was not allowing users to provide their own keys from the UI. Coded tools need to follow a simple authentication pattern to allow for this, see my changes here: |
|
I can add my own key once it's activated for live testing |
aleibovici
pushed a commit
to ASISolutions/LibreChat
that referenced
this pull request
Jan 12, 2025
…#5246) * ✨ feat: Add OpenWeather Tool for Weather Data Retrieval 🌤️ * chore: linting * chore: move test files * fix: tool icon, allow user-provided keys, conform to app key assignment pattern * chore: linting not included in danny-avila#5212 --------- Co-authored-by: Jonathan Addington <[email protected]>
owengo
pushed a commit
to openwengo/LibreChat
that referenced
this pull request
Jan 21, 2025
…#5246) * ✨ feat: Add OpenWeather Tool for Weather Data Retrieval 🌤️ * chore: linting * chore: move test files * fix: tool icon, allow user-provided keys, conform to app key assignment pattern * chore: linting not included in danny-avila#5212 --------- Co-authored-by: Jonathan Addington <[email protected]>
lpirola
pushed a commit
to lpirola/LibreChat
that referenced
this pull request
Jan 22, 2025
…#5246) * ✨ feat: Add OpenWeather Tool for Weather Data Retrieval 🌤️ * chore: linting * chore: move test files * fix: tool icon, allow user-provided keys, conform to app key assignment pattern * chore: linting not included in danny-avila#5212 --------- Co-authored-by: Jonathan Addington <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Originally #5206
Summary
Added OpenWeather integration as a structured tool for LibreChat. This integration provides:
The tool uses OpenWeather's One Call API 3.0, which offers a free tier with 1,500 API calls per month.
This PR demonstrates rapid plugin development using AI assistance - the entire implementation, including comprehensive tests and documentation, was completed in about 3 hours while multitasking.
Features
Change Type
Please delete any irrelevant options.
Testing
npm test -- app/clients/tools/__tests__/openWeather.test.jsIn addition, it is easy enough to test from with LC. I can provide an API key for testing privately.
Documentation
PR submitted at LibreChat-AI/librechat.ai#196